Skip to content

feat: Agent Tasks#3176

Merged
tmilewski merged 8 commits into
mainfrom
tom/user-4863-add-to-clerk-docs
May 19, 2026
Merged

feat: Agent Tasks#3176
tmilewski merged 8 commits into
mainfrom
tom/user-4863-add-to-clerk-docs

Conversation

@tmilewski

@tmilewski tmilewski commented Mar 4, 2026

Copy link
Copy Markdown
Member

@tmilewski tmilewski self-assigned this Mar 4, 2026
@tmilewski tmilewski requested a review from a team as a code owner March 4, 2026 19:02
@vercel

vercel Bot commented Mar 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clerk-docs Ready Ready Preview May 19, 2026 4:26pm

Request Review

Comment thread docs/reference/backend/agent-tasks/revoke.mdx Outdated
Comment thread docs/reference/backend/agent-tasks/create.mdx Outdated
Comment thread docs/guides/development/testing/agent-tasks.mdx
Comment thread docs/reference/backend/agent-tasks/revoke.mdx
Comment thread docs/guides/development/testing/overview.mdx Outdated
Comment thread docs/reference/backend/agent-tasks/create.mdx Outdated
Comment thread docs/reference/backend/overview.mdx
tmilewski and others added 2 commits March 6, 2026 11:25
Co-authored-by: Michael Novotny <manovotny@gmail.com>
Co-authored-by: Michael Novotny <manovotny@gmail.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@manovotny

manovotny commented May 19, 2026

Copy link
Copy Markdown
Contributor

Pushed some changes directly in 020862d.

  • Fixed the revoke example ID prefix from agtask_xxx to agttsk_xxx to match the actual IDPAgentTask constant in BAPI
  • Updated the AgentTask create() response shape to use agentTaskId instead of the deprecated taskId
  • Switched the reference page examples from clerk to clerkClient so the guide and reference pages use the same variable
  • Added the missing ## Backend API (BAPI) endpoint sections to the create and revoke reference pages
  • Dropped console.log(agentTask.url) from the guide and reference examples since the URL is a one-time sign-in link
  • Tightened the permissions wording — BAPI currently rejects anything other than '*', so the docs now say it's the only supported value

Let me know what you think -- specifically on the ID prefix and '*' permission messaging. I want to make sure I'm understanding what I'm seeing on other repos correctly.

Comment thread docs/manifest.json
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@tmilewski

Copy link
Copy Markdown
Member Author

@manovotny TY! My only call-out is the removal of the console.log. I'm less concerned about the console.log specifically, and more about indicating that .url can/should be used to authenticate.

Pushed some changes directly in 020862d.

  • Fixed the revoke example ID prefix from agtask_xxx to agttsk_xxx to match the actual IDPAgentTask constant in BAPI
  • Updated the AgentTask create() response shape to use agentTaskId instead of the deprecated taskId
  • Switched the reference page examples from clerk to clerkClient so the guide and reference pages use the same variable
  • Added the missing ## Backend API (BAPI) endpoint sections to the create and revoke reference pages
  • Dropped console.log(agentTask.url) from the guide and reference examples since the URL is a one-time sign-in link
  • Tightened the permissions wording — BAPI currently rejects anything other than '*', so the docs now say it's the only supported value

Let me know what you think -- specifically on the ID prefix and '*' permission messaging. I want to make sure I'm understanding what I'm seeing on other repos correctly.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@manovotny

Copy link
Copy Markdown
Contributor

Good call. The reason I dropped the console.log is that agentTask.url is credential-equivalent — one visit creates an authenticated session, so logging it to stdout risks leaking it through terminal history or CI output. But you're right that the example still needs to surface .url.

Compromise in 54c27d7 — added a trailing comment naming .url so readers know what to use, without putting it in logs.

@tmilewski

Copy link
Copy Markdown
Member Author

Good call. The reason I dropped the console.log is that agentTask.url is credential-equivalent — one visit creates an authenticated session, so logging it to stdout risks leaking it through terminal history or CI output. But you're right that the example still needs to surface .url.

Compromise in 54c27d7 — added a trailing comment naming .url so readers know what to use, without putting it in logs.

LGTM!

@tmilewski tmilewski merged commit 76489a4 into main May 19, 2026
8 checks passed
@tmilewski tmilewski deleted the tom/user-4863-add-to-clerk-docs branch May 19, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants